Skip to content

fix: switch Prow clone_uri to HTTPS to resolve permission error #28

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
clubanderson merged 1 commit into from
Jan 5, 2026
Merged

fix: switch Prow clone_uri to HTTPS to resolve permission error #28

clubanderson merged 1 commit into from
Jan 5, 2026

Conversation

@shivansh-gohem
Copy link
Contributor

@shivansh-gohem shivansh-gohem commented Dec 27, 2025

Fixes kubestellar/kubestellar#3552

Description
This PR updates the Prow configuration to use HTTPS cloning (https://github.com/...) instead of SSH. This resolves the Permission denied (publickey) error caused by missing SSH secrets in the CI environment.

This implements the solution suggested by @elskow in the issue discussion.

Changes

  • Updated clone_uri in .prow.yaml to use HTTPS.
  • Updated clone_uri in prow/jobs/kubestellar/infra/infra-postsubmits.yaml to use HTTPS.

@kubestellar-prow kubestellar-prow bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Dec 27, 2025
@kubestellar-prow
Copy link
Contributor

kubestellar-prow bot commented Dec 27, 2025

Hi @shivansh-gohem. Thanks for your PR.

I'm waiting for a kubestellar member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@kubestellar-prow kubestellar-prow bot added needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Dec 27, 2025
@shivansh-gohem
Copy link
Contributor Author

shivansh-gohem commented Dec 28, 2025

/cc @MikeSpreitzer could you please review!

@shivansh-gohem
Copy link
Contributor Author

shivansh-gohem commented Jan 5, 2026

@MikeSpreitzer @clubanderson could you please review!!

@clubanderson
Copy link
Contributor

clubanderson commented Jan 5, 2026

/lgtm
/approve

@clubanderson
Copy link
Contributor

clubanderson commented Jan 5, 2026

Thank you @shivansh-gohem - we appreciate your help

@clubanderson
Copy link
Contributor

clubanderson commented Jan 5, 2026

/ok-to-test

@kubestellar-prow kubestellar-prow bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Jan 5, 2026
@clubanderson clubanderson merged commit 4e906bf into kubestellar:main Jan 5, 2026
1 of 2 checks passed
@kubestellar-prow
Copy link
Contributor

kubestellar-prow bot commented Jan 5, 2026

@shivansh-gohem: Updated the job-config configmap in namespace prow at cluster default using the following files:

  • key prow-jobs-kubestellar-infra-infra-postsubmits.yaml using file prow/jobs/kubestellar/infra/infra-postsubmits.yaml
Details

In response to this:

Fixes kubestellar/kubestellar#3552

Description
This PR updates the Prow configuration to use HTTPS cloning (https://github.com/...) instead of SSH. This resolves the Permission denied (publickey) error caused by missing SSH secrets in the CI environment.

This implements the solution suggested by @elskow in the issue discussion.

Changes

  • Updated clone_uri in .prow.yaml to use HTTPS.
  • Updated clone_uri in prow/jobs/kubestellar/infra/infra-postsubmits.yaml to use HTTPS.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@github-actions github-actions bot mentioned this pull request Jan 5, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

dco-signoff: yes Indicates the PR's author has signed the DCO. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

bug: Prow fails to build builder container images

2 participants

@shivansh-gohem @clubanderson